-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Tweak] Gamemode's weights #981
Conversation
Обзор измененийWalkthroughВ файле конфигурации Changes
Poem
Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (1)
Resources/Prototypes/secret_weights.yml (1)
4-13
: Предложение по улучшению читаемости конфигурацииДля лучшего понимания и поддержки конфигурации предлагаю:
- Добавить комментарии, объясняющие назначение каждого режима
- Сгруппировать режимы по категориям (например, командные, индивидуальные, PvE)
- Добавить документацию по методике подбора весов
Пример структуры:
- type: weightedRandom id: Secret weights: # Team-based modes Nukeops: 0.14 # Nuclear operatives team vs station Revolutionary: 0.10 # Revolutionaries vs heads of staff # Individual antagonist modes Traitor: 0.35 # Classic traitor mode # PvE modes Zombie: 0.10 # Zombie outbreak Survival: 0.05 # Station survival # Special events KesslerSyndrome: 0.01 FleshCult: 0.15 Blob: 0.10 # Disabled modes # Changeling: 0.20 # Changelings # Traitorling: 0.10 # Changelings
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/secret_weights.yml
(1 hunks)
🔇 Additional comments (1)
Resources/Prototypes/secret_weights.yml (1)
4-4
: Проверьте влияние изменений на баланс игры
Значительное снижение весов для режимов Nukeops, Revolutionary и Survival может существенно повлиять на частоту их появления:
- Nukeops: снижение на 25%
- Revolutionary: снижение на 33%
- Survival: снижение на 50%
Рекомендуется:
- Провести тестирование с новыми весами
- Собрать отзывы игроков о новом балансе
- Подготовить план отката изменений в случае негативной реакции сообщества
Also applies to: 9-9, 12-12
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
выглядит убедительно, но можете выполнить исшуй кодреббита? уменьшите чтонибудь на 0.1 в секрете
Нужно добавить либо ExtendedThief (воры) либо вампиров в секрет ещё |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Resources/Prototypes/secret_weights.yml (2)
7-7
: Уточните статус режима ZombieteorsРежим закомментирован со значением 0.01. Предлагаю:
- Либо полностью удалить строку, если режим больше не планируется использовать
- Либо добавить комментарий с объяснением, почему режим отключен и когда планируется его возвращение
4-13
: Рассмотрите добавление новых режимовВ обсуждении PR предложено добавить режимы
ExtendedThief
илиVampire
. Предлагаю:
- Создать отдельную задачу для оценки возможности добавления новых режимов
- При добавлении новых режимов пересмотреть все веса для сохранения баланса
Это поможет разнообразить игровой процесс, как того хочет сообщество.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/secret_weights.yml
(1 hunks)
🔇 Additional comments (2)
Resources/Prototypes/secret_weights.yml (2)
4-13
: Сумма весов корректна!
Текущая сумма всех весов равна 1.00, что является математически правильным для распределения вероятностей.
4-4
: Проверьте баланс игровых режимов
Значительное снижение весов для активных режимов (Nukeops
, Revolutionary
, Survival
) может привести к менее динамичному игровому процессу. Рекомендуется провести тестирование с новыми весами для оценки влияния на вовлеченность игроков.
Also applies to: 9-9, 12-12
Описание PR
People are boring with survival, yoooo!
И цифры подправил, да.
Тип PR
Изменения
🆑 Warete
Summary by CodeRabbit
Обновление
Nukeops
,Traitor
,Revolutionary
,FleshCult
,Survival
.Nukeops
: 0.20 → 0.15Traitor
: 0.35 → 0.24Revolutionary
: 0.15 → 0.10FleshCult
: 0.15 → 0.10Survival
: 0.10 → 0.05Zombieteors
закомментирован.ExtendedThief
иVampires
.